Skip to content

Updated ITSTrackSimulation to run with O2DPG - #1115

Merged
iravasen merged 2 commits into
AliceO2Group:masterfrom
IsakovAD:IsakovAD_MCTracksO2DPG
Mar 29, 2022
Merged

Updated ITSTrackSimulation to run with O2DPG#1115
iravasen merged 2 commits into
AliceO2Group:masterfrom
IsakovAD:IsakovAD_MCTracksO2DPG

Conversation

@IsakovAD

Copy link
Copy Markdown
Collaborator
  1. MCKinematics via MCKinematicsReader (previously was done using the .root file provided by o2-MC simulation workflow)
  2. Improved aggregation of the averaged plots
  3. Removed not used parameters from the .json file
  4. Fixed titles of the histograms
  5. Added binominal errors for ratio of the histograms
  6. Added TH1 for pT distribution of primary generated/reconstructed particles

Comment thread Modules/ITS/src/ITSTrackSimTask.cxx Outdated
Comment thread Modules/ITS/src/ITSTrackSimTask.cxx Outdated
@iravasen
iravasen marked this pull request as ready for review March 28, 2022 17:28
@iravasen

Copy link
Copy Markdown
Collaborator

@IsakovAD thanks a lot! I marked this PR as ready (was draft) and I have only the two comments above.

@Barthelemy

Copy link
Copy Markdown
Collaborator

if you want this PR to be in the next FLP Suite please merge it by tomorrow noon.

@iravasen
iravasen enabled auto-merge (squash) March 29, 2022 21:18
@iravasen

Copy link
Copy Markdown
Collaborator

All suggestions done. Enabling auto-merge. Thanks a lot!
@Barthelemy should be automatically merged by noon.

@iravasen
iravasen merged commit 1ee64f1 into AliceO2Group:master Mar 29, 2022
nasirmehdimalik pushed a commit to nasirmehdimalik/QualityControl that referenced this pull request Apr 18, 2022
* Updated ITSTrackSimulation to run with O2DPG

* returned task parameters, added AngularDistribution of track histogram

Co-authored-by: JianLIUhep <jian.liu@cern.ch>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants